Skip to content

Conversation

@gavinking
Copy link
Member

  1. improve/cleanup logging (remove some log messages that don't naively seem more useful than lots of things we don't log)
  2. report problems in AttributeConverter application using an error instead of a debug-level log message
  3. various misc minor code cleanups in the Binders

[Please describe here what your change is about]


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


@hibernate-github-bot
Copy link

hibernate-github-bot bot commented Dec 30, 2024

Thanks for your pull request!

This pull request does not follow the contribution rules. Could you have a look?

❌ All commit messages should start with a JIRA issue key matching pattern HHH-\d+
    ↳ Offending commits: [7901499, 27b34b8, a5b3cd6, b5ec85c, fb81be4, f31d90b]

› This message was automatically generated.

1. improve/cleanup logging (remove some log messages that don't naively
   seem more useful than lots of things we don't log)
2. report problems in AttributeConverter application using an error
   instead of a debug-level log message
3. various misc minor code cleanups in the Binders
@gavinking gavinking force-pushed the mappingloggingandconverters branch from 2ab929a to 7901499 Compare December 30, 2024 16:17
@gavinking gavinking force-pushed the mappingloggingandconverters branch from 9780e65 to 27b34b8 Compare December 30, 2024 19:57
@gavinking gavinking force-pushed the mappingloggingandconverters branch from f74fc00 to f31d90b Compare December 30, 2024 23:57
@gavinking gavinking merged commit 95d5246 into hibernate:main Dec 30, 2024
18 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant